feat: auto failover APIs with LK Cloud#686
Conversation
retries in alternative datacenters on 5xx and transport failures
🦋 Changeset detectedLatest commit: e6ef3f7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Add auto failover APIs for LK Cloud in livekit-server-sdk.
|
CI depends on livekit/livekit#4627 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
| return { origins, ttl }; | ||
| } | ||
|
|
||
| function parseMaxAge(cacheControl: string | null): number { |
There was a problem hiding this comment.
nit: would be nice to have some unit tests for this (can add this in a follow up)
| * cached list when available, otherwise an empty list. Forwards `headers` so a | ||
| * valid token — and any test directives — reach the discovery endpoint. | ||
| */ | ||
| export async function regionOrigins(origin: URL, headers: unknown): Promise<string[]> { |
There was a problem hiding this comment.
might be worth wrapping this in a origin keyed mutex so that we ensure only one regionOrigins request is processed at a time.
For the ones that are queued up afterwards they should be resolved immediately after as long as there's at least some ttl set for the cache.
2 participants
retries in alternative datacenters on 5xx and transport failures
also removed legacy camel-case, which was not needed since we switched to protobuf-es